-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Infineon]Add All-Clusters example app for P6 platform #11300
[Infineon]Add All-Clusters example app for P6 platform #11300
Conversation
Issue #3841connectedhomeip/examples/all-clusters-app/p6/src/ZclCallbacks.cpp Lines 76 to 86 in d9085a9
This comment was generated by todo based on a
|
implement any additional Cluster Server init actionsconnectedhomeip/examples/all-clusters-app/p6/src/ZclCallbacks.cpp Lines 85 to 86 in d9085a9
This comment was generated by todo based on a
|
PR #11300: Size comparison from 2fe9024 to d9085a9 Increases above 0.2%:
Increases (1 build for p6)
Decreases (2 builds for esp32, p6)
Full report (37 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
@praveenCY Is the .bss increase for lock-app expected? |
Yes.. We have updated lwipopts.h to use different PBUF_POOL_SIZE size and this is the reason for increase in BSS size |
PR #11300: Size comparison from 2fe9024 to daf60c4 Increases above 0.2%:
Increases (28 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (3 builds for efr32, esp32, p6)
Full report (37 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Fast tracking, given platform vendor adding tests to their own platform. @praveenCY Can you fix the conflicts please? |
src/app/clusters/barrier-control-server/barrier-control-server.cpp
Outdated
Show resolved
Hide resolved
daf60c4
to
75fe851
Compare
PR #11300: Size comparison from cb7c854 to 75fe851 Increases above 0.2%:
Increases (5 builds for esp32, linux, mbed, p6)
Decreases (4 builds for esp32, linux, p6)
Full report (36 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg)
|
75fe851
to
85b75ce
Compare
…11300) * [Infineon]Add All-Clusters example app for P6 platform * Address code review comments on zcl callbacks
Problem
Add all-clusters example app for Infineon P6 platform
Add LWIP Stable-2.1.X branch for P6 compile
Change overview
Added Infineon P6 All-clusters app files
Added Infineon P6 CI workflow support and app to vscode.json
Added LWIP Stable 2.1.X branch as git submodule for P6 compile
src/inet/IPAddress.cpp change required as latest LWIP has other variables in structure ip6_addr
Below file changes to address compile error like
../../examples/all-clusters-app/p6/third_party/connectedhomeip/src/app/util/util.cpp:423:27: error: format '%x' expects
argument of type 'unsigned int', but argument 4 has type 'chip::CommandId' {aka 'long unsigned int'} [-Werror=format=]
src/app/clusters/barrier-control-server/barrier-control-server.cpp
src/app/clusters/door-lock-server/door-lock-server-core.cpp
src/app/util/util.cpp
Testing
Build and run examples manually
Logs for all-Clusters P6 app commissioning using CHIPTOOL attached
All_Clusters_P6LOG.txt